Skip to content

Add --ignore-default-args flag#1158

Open
ShivanKaul wants to merge 3 commits intovercel-labs:mainfrom
ShivanKaul:ignore-default-args
Open

Add --ignore-default-args flag#1158
ShivanKaul wants to merge 3 commits intovercel-labs:mainfrom
ShivanKaul:ignore-default-args

Conversation

@ShivanKaul
Copy link
Copy Markdown

@ShivanKaul ShivanKaul commented Apr 6, 2026

Fix #682

My use-case is to control a Chromium browser with working component updates. Component updater is currently hard-disabled in agent-browser code.

Playwright allows you to ignoreDefaultArgs, allowing users to remove specific hardcoded Chrome flags from build_chrome_args(). There should be a similar option for agent-browser.

This new flag is available via CLI flag, AGENT_BROWSER_IGNORE_DEFAULT_ARGS env var, and config file (ignoreDefaultArgs). Plumbed through daemon env forwarding and launch hash for relaunch detection.

Note: I used Claude Code to assist with the PR. Happy to work through any feedback. This shouldn't be a large change.

…lt Chrome flags

Mirrors Playwright ignoreDefaultArgs API, allowing users to remove
specific hardcoded Chrome flags from build_chrome_args(). Supports
exact match (--disable-sync) and prefix match (--disable-features
also removes --disable-features=Translate).

Available via CLI flag, AGENT_BROWSER_IGNORE_DEFAULT_ARGS env var,
and config file (ignoreDefaultArgs). Plumbed through daemon env
forwarding and launch hash for relaunch detection.
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 6, 2026

@ShivanKaul is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@ShivanKaul ShivanKaul changed the title Add --ignore-default-args flag to selectively remove default Chrome flags Add --ignore-default-args flag to mirror Playwright Apr 6, 2026
@ShivanKaul ShivanKaul changed the title Add --ignore-default-args flag to mirror Playwright Add --ignore-default-args flag Apr 6, 2026
@ctate
Copy link
Copy Markdown
Collaborator

ctate commented Apr 7, 2026

Thanks for the contribution, @ShivanKaul!

Two small issues to fix before merge:

  1. Run cargo fmt to fix the 3 formatting issues
  2. Add the new flag to skills/agent-browser/SKILL.md per the project's documentation requirements in AGENTS.md

@ShivanKaul
Copy link
Copy Markdown
Author

@ctate thanks for the review! Addressed your comments.

@ShivanKaul
Copy link
Copy Markdown
Author

@ctate gentle ping for this! Any other thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: expose ignoreDefaultArgs option to allow Brave Shields to work

2 participants